Skip to content

Improve type annotation of workflow management methods#22637

Merged
mvdbeek merged 4 commits intogalaxyproject:devfrom
nsoranzo:improve_wf_management_type_annot
May 6, 2026
Merged

Improve type annotation of workflow management methods#22637
mvdbeek merged 4 commits intogalaxyproject:devfrom
nsoranzo:improve_wf_management_type_annot

Conversation

@nsoranzo
Copy link
Copy Markdown
Member

@nsoranzo nsoranzo commented May 5, 2026

Also:

  • Fix mypy exclude config

How to test the changes?

(Select all options that apply)

  • I've included appropriate automated tests.
  • This is a refactoring of components with existing test coverage.
  • Instructions for manual testing are as follows:
    1. [add testing steps and prerequisites here if you didn't write automated tests covering all your changes]

License

  • I agree to license these and all my past contributions to the core galaxy codebase under the MIT license.

@github-project-automation github-project-automation Bot moved this to Needs Review in Galaxy Dev - weeklies May 5, 2026
@github-actions github-actions Bot added area/API area/workflows area/database Galaxy's database or data access layer labels May 5, 2026
@github-actions github-actions Bot added this to the 26.1 milestone May 5, 2026
Comment thread lib/galaxy/managers/workflows.py
Comment thread lib/galaxy/managers/workflows.py
Comment thread lib/galaxy/managers/workflows.py
@nsoranzo nsoranzo force-pushed the improve_wf_management_type_annot branch from 5e3a417 to 85ae996 Compare May 6, 2026 04:35
@mvdbeek mvdbeek merged commit a7ab245 into galaxyproject:dev May 6, 2026
58 of 59 checks passed
@github-project-automation github-project-automation Bot moved this from Needs Review to Done in Galaxy Dev - weeklies May 6, 2026
@mvdbeek
Copy link
Copy Markdown
Member

mvdbeek commented May 6, 2026

Thank you @nsoranzo!

@nsoranzo nsoranzo deleted the improve_wf_management_type_annot branch May 6, 2026 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Development

Successfully merging this pull request may close these issues.

2 participants